chore: Update prettier config for quoted object properties#2099
Merged
BeksOmega merged 2 commits intoRaspberryPiFoundation:masterfrom Dec 7, 2023
Merged
chore: Update prettier config for quoted object properties#2099BeksOmega merged 2 commits intoRaspberryPiFoundation:masterfrom
BeksOmega merged 2 commits intoRaspberryPiFoundation:masterfrom
Conversation
Set the quoteProps rule to consistent in the .prettierrc.js file Run npm run format to automatically fix all properties.
shivalipatel6
requested review from
NeilFraser
and removed request for
a team
…iFoundation#2099 ran npm ci and npm run format to echo changes throughout samples
Contributor
|
For the error you're seeing, can you try running |
BeksOmega
approved these changes
Dec 7, 2023
Contributor
BeksOmega
left a comment
BeksOmega
left a comment
There was a problem hiding this comment.
Sweet this looks great! Thank you for the fixes =)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Set the quoteProps rule to consistent in the .prettierrc.js file Run npm run format to automatically fix all properties.
The basics
The details
Resolves
Setting quotations to properties that must be quoted, and then if any property in an object is quoted, all of them will be to remain consistent.
Fixes #1980
Proposed Changes
Reason for Changes
Test Coverage
Documentation
Additional Information
When I ran npm run format I got this response. I wasn't sure if it was the correct response or not, so I am mentioning in the case that there is a problem that I need to fix.
